-
Notifications
You must be signed in to change notification settings - Fork 282
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Release v1.7.4 #3472
base: master
Are you sure you want to change the base?
Release v1.7.4 #3472
Conversation
Do we merge with the CI failure? |
I don’t think we should. Why do we need a new console release at this time? |
Yes, This PDF issue is not a real failure, PDF is rendered correctly in the UI Locally I see an error related to pdf worker library required for this and testcafe. This needs further investigation |
0caa05e
to
1d8885e
Compare
b0d3d56
to
9033e5f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add Changelog.md
Signed-off-by: Benjamin Perez <[email protected]>
9033e5f
to
286dc98
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bexsoft Did you regenerate the files after rebasing to the latest master
? When I run make release
, then all generated files are different but that may be caused due to different versions. I do think the build
folder should be identical and not depend on whoever runs make release
.
For that reason, I am always a fan of letting the CI/CD build the final output instead. Then it's consistent.
Yes, @ramondeklein, all files are updated with the latest master change. we use a release script to generate these files, maybe could be related |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WIth the removal of functionality, this isn't a patch. It should at least be named v1.8.0. But I don't think version number is actually being used -outside MinIO-, so I guess it's fine...
@bexsoft corresponding docs ready to merge |
No description provided.